Skip to content

Conversation

@mjclemente
Copy link
Contributor

This partially deals with #2035, though does not resolve it entirely.

CFML has independent tag and script based syntaxes. This adds support for script syntax.

Here are some language references:

@github-actions
Copy link

github-actions bot commented Feb 18, 2021

JS File Size Changes (gzipped)

A total of 3 files have changed, with a combined diff of +720 B (+16.3%).

file master pull size diff % diff
components/prism-cfscript.min.js 0 Bytes 699 B +699 B +100.0%
plugins/autoloader/prism-autoloader.min.js 2.19 KB 2.19 KB +10 B +0.5%
plugins/show-language/prism-show-language.min.js 2.23 KB 2.24 KB +11 B +0.5%

Generated by 🚫 dangerJS against 7695c65

Copy link
Member

@RunDevelopment RunDevelopment left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work @mjclemente!

A couple of minor things and then this is ready.

@mjclemente
Copy link
Contributor Author

Thanks for the fast, thorough, and really helpful review 😀

I believe I've made all requested changes.

Copy link
Member

@RunDevelopment RunDevelopment left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the quick response!

One more comment. Also, could you please add the link to https://cfdocs.org/script in a comment at the beginning of the file? This will be helpful to future contributors.

@mjclemente
Copy link
Contributor Author

Makes sense!

Changes made. Again, thanks for the prompt and thorough handling!

@RunDevelopment RunDevelopment merged commit b0a6ec8 into PrismJS:master Feb 19, 2021
@RunDevelopment
Copy link
Member

Thank you for contributing @mjclemente!

@mjclemente
Copy link
Contributor Author

Thank you! 🎉

@dawesi
Copy link

dawesi commented Sep 7, 2025

can we also create a branch of cfml for cfml/lucee (which boxlang also supports that syntax)

This is divergent from Adobe CFML/CFScript now also, as it containers many functions / helpers methods that are not in coldfusion and vice versa.

They publish the latest docs in json files zipped here.

https://docs.lucee.org/lucee-docs-json.zip

@mjclemente
Copy link
Contributor Author

Given that Lucee/Boxlang/Adobe ColdFusion are all CFML, my first thought would be that they could all be incorporated here. While there may be some different tags, I don't see any issues in including them all - they have more in common than different (at least Lucee CFML and Adobe). Not sure what functions are missing at this point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants